revert: bump typescript from 6.0.3 to 7.0.2 (#882) to unbreak main - #890
Conversation
PR #882 auto-merged the dependabot bump of typescript 6.0.3 -> 7.0.2, which broke `main`: the `build` job's lint step crashes because @typescript-eslint/typescript-estree is incompatible with TypeScript 7.0 (`TypeError: Cannot read properties of undefined (reading 'Cjs')` in create-program/shared.js). Note `tsc --noEmit` type-check still passes — the breakage is purely the ESLint tooling, not the source. The merge landed before the `build` check finished (dependabot-auto-merge does not gate on it), so the regression reached main unnoticed and every subsequent merge inherits the red lint. This reverts the version pin across package.json, apps/web/package.json, and package-lock.json back to the last known-green state (^6.0.3). Immediate fix to unbreak main. The durable path is to keep TS7 and bump @typescript-eslint to a TS7-compatible release; until such a version is pinned, dependabot will re-propose this bump. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CPYC3tfxK9YzvmMsBvX6TN
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🔍 PR Validation✅ Current validation passed. |
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository YAML (base), Repository UI (inherited), Organization UI (inherited) Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. OpenSSF Scorecard
Scanned Files
|
Agent Completion Truth Gate: BLOCKEDReasons: Machine-readable verdict{
"details": {
"invalid_fields": [
"issue.number",
"policy.agent_login",
"policy.run_id"
]
},
"reasons": [
"invalid_payload"
],
"verdict": "blocked"
} |
There was a problem hiding this comment.
Pull request overview
Reverts TypeScript 7 to the parser-compatible 6.0.3 release, restoring frontend lint/build functionality.
Changes:
- Restores TypeScript
^6.0.3in both manifests. - Reverts the lockfile and removes TypeScript 7 platform packages.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
package.json |
Restores the root TypeScript version. |
apps/web/package.json |
Restores the web workspace TypeScript version. |
package-lock.json |
Locks TypeScript 6.0.3 and removes TypeScript 7 artifacts. |
Summary
mainis currently red. PR #882 auto-merged the dependabot bump typescript6.0.3→7.0.2at2026-07-19T05:06:58Z, and CI on the resulting merge commit (0383146a) failed — the immediately preceding commit (#881) was green.Root cause is not a type error —
tsc --noEmit(type-check) passes. Thebuildjob's lint step crashes because@typescript-eslint/typescript-estree is incompatible with TypeScript 7.0:The auto-merge (
dependabot-auto-merge.yml) merged before thebuildcheck finished, so the regression reachedmainunnoticed and every subsequent merge (#884, #885, #887, #877, …) inherits the red lint.This PR is a straight
git revertof the bump commit (ebc09e5), restoringtypescriptto^6.0.3acrosspackage.json,apps/web/package.json, andpackage-lock.json— i.e. the last known-green dependency state.+10 / −371, the exact inverse of the bump.This is the immediate unblock. The durable path is to keep TS 7 and bump
@typescript-eslintto a TS7-compatible release; until such a version is pinned, dependabot will re-propose this bump — consider addingbuild/lint as a required check and/or an ignore rule for the TS major until the toolchain catches up.Linked issue
No tracking issue — opened as an incident revert for the broken
main.Verification
agent-completion/truth-gatecheck is independently broken across all open PRs (separate issue) and will show red here regardless.Opened as draft — it targets protected
mainand should merge only on human sign-off.Agent provenance
Automated remediation authored by Claude Code (background PR-shepherd routine). No linked agent-task issue / agent-lock manifest exists for this incident revert, so the provenance manifest is intentionally omitted rather than fabricated.
Generated by Claude Code